edgeoperator: az aldo system-readiness show (preview) - #10205
edgeoperator: az aldo system-readiness show (preview)#10205kleyvaortega wants to merge 7 commits into
Conversation
…iness show Add recording processors to the scenario test to drop private-cloud auth interactions and normalize the ARM host, and sanitize the recording so it contains no tokens or private ALDO endpoints. Playback runs offline. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
|
Hi kleyvaortega, |
|
Azure Pipelines: There may be pipelines that require an authorized user to comment /azp run to run. |
|
Automated sensitive-information remediation ran on this pull request.
If a credential was exposed, rotate or revoke it immediately. Detected values are never copied into this comment. ✅ Keep the redaction · ❌ Dispute the redaction GitHub only supports a fixed reaction set, so 👍 represents ✅ and 👎 represents ❌. The bot-created reactions are only poll choices. Posted by agent-assist (autonomous bug-fix pipeline). |
|
/azp run |
|
Azure Pipelines: Successfully started running 2 pipeline(s). |
|
aldo |
Add MIT license headers to the 7 source/test files that were missing them (Check License gate), and register the `az aldo` command group in src/service_name.json as "ALDO EdgeOperator" (azdev linter/style service name gate). Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Map the az aldo umbrella group to "Azure Local Disconnected Operations" and the EdgeOperator sub-group az aldo system-readiness to "Edge Operator", so future non-EdgeOperator ALDO commands are not mislabeled. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
…into users/kleyvaortega/systemReadinessExtensions # Conflicts: # src/service_name.json
|
/azp run |
|
Azure Pipelines: Successfully started running 2 pipeline(s). |
|
/azp run |
|
Azure Pipelines: Successfully started running 2 pipeline(s). |
|
Azure Pipelines: There may be pipelines that require an authorized user to comment /azp run to run. |
There was a problem hiding this comment.
🟡 Changes recommended
Packaging/service-attribution details need adjustment to avoid misleading wheel metadata and inconsistent command-to-service mapping.
Once you've addressed the issues Copilot identified, you can request another Copilot review.
Pull request overview
This PR introduces a new preview Azure CLI extension (edgeoperator / azext_edgeoperator) that adds the read-only command az aldo system-readiness show, implemented via send_raw_request to query the subscription-scoped singleton Microsoft.EdgeOperator/systemReadiness/default using API version 2026-06-01-preview.
Changes:
- Adds the new
edgeoperatorextension package with command loader, help, command registration, and raw ARM call implementation. - Adds unit + scenario tests (with recording processors and a sanitized recording) to validate URL construction and response shape.
- Updates command-to-service attribution (
src/service_name.json) and adds a CODEOWNERS entry for the new extension folder.
File summaries
| File | Description |
|---|---|
| src/service_name.json | Adds service attribution entries for az aldo and az aldo system-readiness. |
| src/edgeoperator/setup.py | Adds packaging metadata and setuptools configuration for the new extension. |
| src/edgeoperator/setup.cfg | Adds wheel build configuration for the new extension. |
| src/edgeoperator/README.rst | Documents the extension and the system-readiness show command. |
| src/edgeoperator/HISTORY.rst | Introduces initial release notes for version 0.1.0. |
| src/edgeoperator/azext_edgeoperator/custom.py | Implements the system-readiness show raw ARM GET request. |
| src/edgeoperator/azext_edgeoperator/commands.py | Registers aldo system-readiness show as a custom show command. |
| src/edgeoperator/azext_edgeoperator/azext_metadata.json | Marks the extension as preview and sets min CLI core version. |
| src/edgeoperator/azext_edgeoperator/_params.py | Adds (currently empty) argument loader entrypoint. |
| src/edgeoperator/azext_edgeoperator/_help.py | Adds help text for aldo and aldo system-readiness show. |
| src/edgeoperator/azext_edgeoperator/init.py | Adds the extension command loader wiring. |
| src/edgeoperator/azext_edgeoperator/tests/latest/test_system_readiness_scenario.py | Adds a scenario test with recording processors to normalize ARM host and filter private auth traffic. |
| src/edgeoperator/azext_edgeoperator/tests/latest/test_custom.py | Adds a unit test validating singleton URL construction and request dispatch. |
| src/edgeoperator/azext_edgeoperator/tests/latest/recordings/test_system_readiness_show.yaml | Adds a sanitized scenario test recording. |
| src/edgeoperator/azext_edgeoperator/tests/latest/init.py | Adds test package init (new file). |
| src/edgeoperator/azext_edgeoperator/tests/init.py | Adds test package init (new file). |
| .github/CODEOWNERS | Adds code ownership for /src/edgeoperator/. |
Review details
- Files reviewed: 15/17 changed files
- Comments generated: 3
- Review effort level: Lite
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
|
/azp run |
|
Azure Pipelines: Successfully started running 2 pipeline(s). |
Summary
Adds a new preview Azure CLI extension
edgeoperatorexposing the read-only command:az aldo system-readiness show- GET onMicrosoft.EdgeOperator/systemReadiness/default(subscription-scoped singleton), API version2026-06-01-preview.Details
az aldo; extension/packageedgeoperator(azext_edgeoperator).2.75.0; marked preview.send_raw_requestagainst the active cloud's ARM endpoint.302f60196ed947128acabc64b558237750017b80.Tests
test_custom.py) validates URL construction and dispatch.test_system_readiness_scenario.py) validates the public response shape with unifiedproperties.categoriesentries and recording processors that:azdev style,azdev linter,azdev test, andazdev extension buildpassed during local PR validation.Notes for reviewers
send_raw_request; it does not depend on publication of a generated management SDK package.